These changes appear to replace is_useful with is_data || is_text. I think this would not change the meaning, except that is_bss was removed. is_bss may not occur in the cases you looked at but it is a case that we need to be able to handle. A .bss section is a data section where the contents are initialized to zero.

I didn't think I removed is_bss. I only removed is_useful because I noticed that there's no else case for that. Maybe it does make sense to leave it in though. The semantics is the same I think I just thought less code is better. Please feel free to ignore.